Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Piece.xs: eliminate C++ guards #64

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tonycoz
Copy link
Member

@tonycoz tonycoz commented May 22, 2023

These aren't needed with modern perls, and it's only useful to build modern perls as C++.

This was causing an error in a system header when building blead as C++ with MSVC:

C:\Program Files (x86)\Windows Kits\10\include\10.0.22000.0\um\wspiapi.h(53):
error C2894: templates cannot be declared to have 'C' linkage

See Perl/perl5#21073 for more information

These aren't needed with modern perls, and it's only useful to
build modern perls as C++.

This was causing an error in a system header when building blead
as C++ with MSVC:

    C:\Program Files (x86)\Windows Kits\10\include\10.0.22000.0\um\wspiapi.h(53):
    error C2894: templates cannot be declared to have 'C' linkage

See Perl/perl5#21073 for more information
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant